gtk: Avoid using gdk_screen_get_setting

We can get the setting from the display instead.

Add gtk_settings_get_for_display

This will help us avoiding screens in many places.

Stop using the GdkScreen get_setting vfunc

We can now call the GdkDisplay implementation instead.

use the GdkDisplay get_setting vfunc

Instead of calling the GdkScreen implementation, use our
own vfunc now.

x11: Implement the GdkDisplay get_setting vfunc

Just call the GdkScreen vfunc.

wayland: Implement the GdkDisplay get_setting vfunc

Just call the GdkScreen vfunc.

broadway: Implement the GdkDisplay get_setting vfunc

This is just a trivial implementation.

quartz: Implement the GdkDisplay get_setting vfunc

Just call the GdkScreen vfunc.

mir: Implement the GdkDisplay get_setting vfunc

Just call the corresponding screen vfunc.

gdk: Add a get_setting vfunc to GdkDisplay

This will eventually replace the GdkScreen vfunc of the
same name.

Remove GdkScreen api usage

Use GdkDisplay equivalents where they exist.

Store the display inside GtkSettings

This is a step towards dropping GdkScreen.

Remove gdk_setting_get

This is a minor convenience api, and it is unused.

Forgotten files

Clean up the setting event from headers and docs too.

gdk: Drop settings events

We are not emitting these events anymore, so lets remove them
from the api. The GdkSettingAction enum is moved to xsettings-client.c
where its only use remains.

x11: Stop emitting settings events

Not needed anymore.

wayland: Stop emitting setting events

Not needed anymore.

quartz: Stop emitting setting events

Not needed anymore.

mir: Stop emitting setting events

Not needed anymore.

gtk: Stop handling settings events

We don't need this anymore, now that we use the ::setting-changed
signal.

gtk: Connect to GdkDisplay::setting-changed

We still handle the setting event, for now.
That will be removed in a future commit.

x11: Emit GdkDisplay::setting-changed

We still generate the event too.
That will be removed in a future commit.

wayland: Emit GdkDisplay::setting-changed

We still generate the event too.
That will be removed in a future commit.

quartz: Emit GdkDisplay::setting-changed

We still generate the event too.
That will be removed in a future commit.

mir: Emit GdkDisplay::setting-changed

We still generate the event too.
That will be removed in a future commit.

gdk: Add settings api to GdkDisplay

This commit adds gdk_display_get_setting and a ::setting-changed
signal, which will replace the settings event we use now. Note
that I've done away with the GdkSettingAction argument that the
event has, since we are not using it at all.

make GDK_DEBUG=opengl work on win32

input/IME: Defer the emit of the "commit" signal

On Windows, when IME is used, each keystroke results in the
WM_IME_COMPOSITION event being sent first.  This means that in our case
when one decides on to accept the input that is in the preedit buffer,
we first get from Windows the WM_IME_COMPOSITION event
(where we emit the commit signal), followed by the WM_IME_ENDCOMPOSITION
event (where we emit the pair of preedit-changed and preedit-end
signals).

Since commit f11f989 (GtkEntry: Remove recompute idle), we do the input
recomputation directly, this will cause a pair of "Pango-WARNING:
Assertion failed: (index >= 0 && index <= layout->length)" being shown,
as gtkentry.c's priv->preedit_length and priv->preedit_cursor was unable
to be reset to 0 in time as a result of the recomputation triggered by
the commit being done before the reset of priv->preedit_length and
priv->preedit_cursor (which are no longer valid as we essentially say
that we are done with the preedit buffer).

As we could only acquire the final string that was entered in this
preedit session when we handle the WM_IME_COMPOSITION event, fix this by
saving up the final string we acquire from Windows IME in UTF-8 when we
handle the WM_IME_COMPOSITION event from Windows, and emit the commit
signal with that string after we emit the preedit-changed and
preedit-end signals when we handle the WM_IME_ENDCOMPOSITION event from
Windows, which comes afterwards.

Also fix the formatting of the code around the parts of the files that
was changed.

https://bugzilla.gnome.org/show_bug.cgi?id=787142

cssimagescaled: Remove now unused struct member

gdk: Remove leftovers from GdkScreen::size-changed removal

inspector: Use the new monitor change signals

And on't use the GdkScreen anymore now that the GdkDisplay provides all
the information we care about.

screen: Remove unused vfuncs

widget: Make parent property read-only

widget: Don't notify :parent when rearranging widgets

gtk_widget_reposition_after is also valid to call if the widget already
has a parent, so don't notify the parent property in that case.

widget: Remove parent-set signal

When a widget unparents its child widget manually in finalize, this can
lead to the parent-set signal being emitted for those child widgets. The
parent already has a ref_count of 0 though, so it can't be used in a
meaningful way. Specifically, emitting the signal will already try to
ref the parent which prints a critical.

Since GtkWidget already has a "parent" property, one can use its notify
signal instead to get notified when the parent widget changes.

inspector: Remove node != NULL check from recorder model

We use NULL to remove the selection.

checkbutton: Destroy indicator widget when draw-indicator is FALSE

cssimagescaled: Just return the computed image

... instead of returning either itself with uncomputed images or a
copy of itself with only one computed image and lots of other
uncomputed images that we're never gonna look at again.

This fixes expressions like -gtk-scaled(-gtk-recolor(...),-gtk-recolor(...))
which Adwaita uses for checkmarks and bullets.

Avoid creating trivial shadow nodes

The Vulkan renderer creates a fallback surface for each shadow
node, even if we end up not rendering anything to it. Avoiding
this is a nice optimization.

Adwaita: Use scaled icons for checks and bullets

This gives us crip rendering on hidpi screens.

Adwaita: Add scaled version of bullets and checks

We use these assets as .symbolic.png now, so for hidpi, we
should provide @2 variants.

gsk: Apply scale factor for fallback rendering

This fixes blurry text and icons whenever we apply shadows
in a hidpi window. Shadow nodes are the last ones that we
still use fallback for, and this was causing us to render
the text blurry.

Triival cleanup

No need to go to the window, we store a copy of the scale factor
in the render pass object.

inspector: Show details for shadow nodes

One of the few node types that we're not detailing yet.

gsk: Scale glyphs in the glyph cache

Pass a scale factor when caching glyphs or looking them
up in the cache. The glyphs in the cache are rendered
with subpixel precision determined by the scale. Update
all callers to pass a scale factor according to the window
scale. This lets us render crisp glyphs on hidpi systems.

vulkan: Handle changing window scale

The code that checks for the proper size of the our swapchain
was not taking window scale fully into account. With this change,
setting the window scale to 2 in the inspector causes the window
to grow and rendering to be scaled up as expected, with Vulkan,
in the same way it already is with cairo.

gsk: Drop the GskRenderer::scale-factor property

This is can always be obtained from the window that is already
associated with the renderer, no need to maintain a separate
property for it.

x11: Query whether we have GLX support

Epoxy 1.4 has new ad hoc API that we can use to check whether GLX is
available on the current system.

If we didn't use this API, we'd have to manually dlopen libGL (or its
equivalent on different OSes) and check if it had GLX symbols; since
Epoxy already does all of this internally, we can simply ask it instead.

https://bugzilla.gnome.org/show_bug.cgi?id=775279

Bump up the dependency on libepoxy

We are going to use new API to detect whether or not GLX is available
before calling GL API.

https://bugzilla.gnome.org/show_bug.cgi?id=775279

Adwaita: Add back visible focus outlines for list rows

sizerequest: Don't pass uninitialized values to measure

The GtkWidgetClass::measure vfunc is not required to assign a value to
any of the (out) parameters, so we need to initialize the locals we pass
to it, otherwise we can end up with a garbage size request.

GtkPathBar: Centralize handling of outstanding cancellables

The path bar would crash if we disposed it before all pending I/O
operations had finished.  Now we remember all the outstanding
operations directly in the GtkPathBarPrivate, and deal with them
consistently.

gsk: Make text nodes more compact

The copy of the PangoGlyphString we do here was showing up
in some profiles. To avoid it, allocate the PangoGlyphInfo array
as part of the node itself. Update all callers to deal with
the slight api change required for this.

wayland: scale down reported monitor geometry

According to the documentation, gdk_monitor_get_geometry() reports the
monitor geometry in ”application pixels”, not in ”device pixels”,
meaning that the actual device resolution needs to be scaled down by the
scale factor of the output.

x11 backend does that downscaling, whereas Wayland backend did not,
causing a discrepancy depending on the backend used.

https://bugzilla.gnome.org/show_bug.cgi?id=783995

GtkHeaderBar: Reconsider buttons if any tiling state changes

Looking at the 1-bit "tiled or not?" state is not necessarily
enough.

Signed-off-by: Simon McVittie <smcv@debian.org>
https://bugzilla.gnome.org/show_bug.cgi?id=789357

Wayland: Implement KDE's SSD protocol

If the compositor prefers server-side decorations and the client doesn't
customize the title bar, we disable client-side decorations and let the
compositor know. Otherwise, we continue to use client-side decorations.

Signed-off-by: Drew DeVault <sir@cmpwn.com>
https://bugzilla.gnome.org/show_bug.cgi?id=781909

wayland: Do not constrain saved window size

Under Wayland, an xdg_surface.configure with size 0x0 means it's up to
the client to set its size.

When transitioning from maximized state to un-maximized, the Wayland
compositor will send such an 0x0 configure so that the client can
restore its original size.

However, the original size was already constrained, so re-applying
size constrains can lead to a smaller size when using size increments.

Avoid this caveat by not applying size constrains when we are restoring
the original size.

https://bugzilla.gnome.org/show_bug.cgi?id=777072

appchooserwidget: Don't limit application list unconditionally

As documented, GtkAppChooser is "typically [used] for the purpose of
opening a file". However given that applications that support neither
opening files nor URLs are filtered out, the chooser is not actual
useful for any other (atypical) usage. Change that by only applying
the filtering if a content-type was set, and use the full unfiltered
list otherwise.

https://bugzilla.gnome.org/show_bug.cgi?id=789327

actionable: Allow all detailed action name formats

Use g_action_parse_detailed_name() to enable use of this API
for actions with non-string parameter.

https://bugzilla.gnome.org/show_bug.cgi?id=788841

printing: Install printbackends

Add "install : true" for each printbackend so that they are installed.

https://bugzilla.gnome.org/show_bug.cgi?id=789185

wayland: Don't spew warnings for blank cursors

We were unnecessarily spewing warnings when blank cursors
were getting a new scale set. Standardize on "none" as the
name for blank cursors, and avoid the warning.

https://bugzilla.gnome.org/show_bug.cgi?id=775217

icon-browser: Make non-symbolic icons visible again

The commit that added the scalable size to the details dialog
inadvertedly caused all icons to be hidden. No need to do that.

Drop gtk_cell_renderer_get_size

This function has been deprecated since 3.0, time to drop it.

Drop unused includes of gtkrender.h

We don't use the gtk_render apis internally anymore.
Drop these includes so it is clear where the remaining
uses are.

Some minimal theme fixes for focus drawing

We don't want focus rectangles everywhere, so set them up
explicitly for the widgets where we want them, and only
if focus(visible) is set.

Remove a shortcut in gtk_widget_set_focus_child

Since focus can now be represented by more than one state,
just looking at the focus_child is no longer sufficient - we
may fail to propagate :focus(visible) if we do so. For now,
just remove the shortcut and always do the work.

Always draw outline

We can now control with CSS where there the outline
is drawn.

Support the new state in CSS selectors

Make :focus(visible) match the new state.

Set the new state flag

For now, we only set the new visible focus state
on the focus widget, when we have visible focus.
Later on, we will allow setting it on other widgets.

Add a state flag for visible focus

The new flag is called GTK_STATE_FLAGS_FOCUS_VISIBLE.

gdk: Clear GL context when window is withdrawn

Some clients (e.g. gnome-online-accounts) quickly unmap and map
a window. With some backends the backend surface will be replaced
causing the application to crash because the GL context is still
using the old surface. Clearing the GL context when a window is
withdrawn fixes this.

https://bugzilla.gnome.org/show_bug.cgi?id=789141

Adwaita: Remove focus outlines from scrollbars and scales

gtk3 didn't have focus outlines for these, so remove them for now.

Sorry lapo.

gtk/gtkcssenumvalue.c: Deal with __builtin_popcount on MSVC

__builtin_popcount is a GCCism that is used to count the number of bits
involved, which means any non GCC/CLang compilers won't like the code,
meaning that on MSVC builds we must implement it ourselves.

We first use __cpuid() to check whether the CPU supports the popcount
instruction, if it does, we use the __popcnt intrinsic, otherwise
(untested, since I don't have a system that does not have the
instruction), we use the suggested hacks at
http://graphics.stanford.edu/~seander/bithacks.html#CountBitsSetParallel

https://bugzilla.gnome.org/show_bug.cgi?id=773299

testsuite/gsk/test-render-nodes.c: Include <stdlib.h>

This is so that the compiler won't complain about our use of exit()

https://bugzilla.gnome.org/show_bug.cgi?id=773299

gtk/gtkwin32theme.c: Include gdk/gdkprivate.h

We still need to access the GdkEvent structure here directly, as using
the GdkEvent getters is likely not worth the trouble involved.

Please see Emmanuele's comment (#97) of the following bug URL.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

gdk/win32/gdkwindow-win32.c: Fix gdk_win32_window_set_icon_list()

The list of surfaces passed into the function may be NULL, so don't try
to initialize the surfaces if it is so, to avoid a crash.

Also, remove the cast to GdkPixbuf* for getting surfaces->data, as we
are already using a cairo_surface_t*.

https://bugzilla.gnome.org/show_bug?id=773299

modules/input/gtkimcontextime.c: Fix build

The GdkEvent structure became opaque, so we need to update the code so
that we use the getters and setters here.

https://bugzilla.gnome.org/show_bug?id=773299

Fix build

Don't do two things at once and forget to run ninja before pushing.

Add default return values to switch statements

We are using g_assert_not_reached() without doing anything, assuming it
aborts the program. In release builds however, it is ignored.

Add a return value

When building with G_DISABLE_ASSERT, the g_assert_not_reached()
statement won't do anything, and we're going to fall through, and the
compiler will emit a warning that we're not returning anything from a
function with a return value.

inspector: Show more render node details

Should clip and container node details.

gdk/win32/gdkwindow-win32.c: Fix build

Make up for the missed commas...

vulkan: No need to redefine gl_PerVertex

The compiler has those predefined, so use them.

vulkan: Delete unused shaders

These have been renamed to .frag/.vert, apparently the originals weren't
deleted.

Mention pixbufs in the migration guide

They are being phased out.

Drop all uses of GdkPixbufAnimation in the gtk APIs

These are basically animated gifs, and don't fit well in how
modern things animate.

build: Warn about ignored qualifiers

https://bugzilla.gnome.org/show_bug.cgi?id=789351

Remove unnecessary const

We return a scalar value, so we don't need it to be constant.

https://bugzilla.gnome.org/show_bug.cgi?id=789351

GtkIconHelper: Remove unused code

Drop pixbuf support in IconHelper and ImageDefinition

These are no longer used, instead we always covert to surface as
early as possible and drop the pixbuf.

This means we never store both the pixbuf and the surface at
for any longer time, which is wasteful. Also, its one step further
to drop GdkPixbufs from generic use in our APIs.

GtkDragSource: Don't use gtk_image_definition_new_pixbuf

We want to get rid of pixbuf image definitions.

Long term this should probably be gtk_drag_source_set_icon_surface instead.

CellRendererPixbuf: Never store pixbufs

Rather than store the pixbufs as themselves we immediately convert
them to surfaces. In the uncommon case that a pixbuf is read back
from the renderer we generate a new one from the surface data.

GtkImage: Drop support for storing pixbufs

This drops the pixbuf property and the pixbuf getters. We keep
gtk_image_new/set_from_pixbuf, but these are small helpers that
immediately convert to a surface, and there is no way to later get
back the pixbuf you passed in.

The from file/resource codepaths are also changed to load a surface
instead of a pixbuf.

window: Use correct destroy notify for icon list

The list contains cairo_surface_t instances now, not GdkPixbuf anymore.

icontheme: Silence some compiler warnings

dnd: Add more surface apis

The new function, gtk_drag_source_set_icon_surface, is for setting
a drag source icon without using GdkPixbuf.

GtkWindow/GdkWindow: Finish converting icons to surfaces

There were some parts left, for instance gdk_window_set_icon_list.

GtkEntry: Use surfaces, not pixbufs for bitmapped icons

aboutdialog: Use set_boxed on surface

cairo_surface_t is boxed, not a GObject.

inspector: show color matrix node details

Show the matrix and the offset for color matrix nodes.